Skip to content

Add lhl2617/VSLilyPond suite#752

Draft
lbssousa wants to merge 2 commits intoEclipseFdn:masterfrom
lbssousa:vslilypond-formatter
Draft

Add lhl2617/VSLilyPond suite#752
lbssousa wants to merge 2 commits intoEclipseFdn:masterfrom
lbssousa:vslilypond-formatter

Conversation

@lbssousa
Copy link
Contributor

@lbssousa lbssousa commented Mar 29, 2024

  • I have read the note above about PRs contributing or fixing extensions
  • I have tried reaching out to the extension maintainers about publishing this extension to Open VSX (if not, please create an issue in the extension's repo using this template).
  • This extension has an OSI-approved OSS license (we don't accept proprietary extensions in this repository)

Description

This PR adds https://github.com/lhl2617/VSLilyPond suite to the extension marketplace

@filiptronicek
Copy link
Collaborator

filiptronicek commented Apr 2, 2024

Hi there @lbssousa. Could you please look into the logs of the failed CI and convert this to a draft in the meantime if this takes some more effort to set up?

@lbssousa lbssousa marked this pull request as draft April 2, 2024 18:50
@lbssousa
Copy link
Contributor Author

lbssousa commented Apr 2, 2024

I'm trying to debug CI errors. Reading the logs, I've found it's failing at this point, when trying to bundle python-ly for extension VSLilyPond-formatter:

$ staticx /tmp/repository/main/ly/dist-linux/python-ly /tmp/repository/main/ly/dist-linux/python-ly.static
Traceback (most recent call last):
  File "ly/build.py", line 71, in <module>
    cmd(f"staticx {bin_path} {bin_path}.static")
  File "ly/build.py", line 31, in cmd
    raise Exception(
Exception: Command 'staticx /tmp/repository/main/ly/dist-linux/python-ly /tmp/repository/main/ly/dist-linux/python-ly.static' exited with code 2, expected 0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@lbssousa @filiptronicek